Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: availability update #199

Merged
merged 1 commit into from
Oct 11, 2023
Merged

fix: availability update #199

merged 1 commit into from
Oct 11, 2023

Conversation

ruilopesm
Copy link
Member

No description provided.

@reviewpad
Copy link

reviewpad bot commented Oct 11, 2023

AI-Generated Summary: This pull request includes changes made to the EventPage component in the file apps/app/pages/event/[id].tsx. The purpose of the changes is to fix updating of availabilities, specifically focusing on simplifying and refactoring the code for better efficiency and readability.

The updates included removing unused states such as 'availabilities' and 'mentors', and optimizing the useEffect hook that was connected with the 'availabilities'. In addition, modifications were made to the isMentorAlreadyRegistered and changeMentorAvailability functions, which now use the refactored list of mentors instead of 'availabilities'. The overall changes have resulted in a net reduction of lines of code enhancing maintainability and overall performance.

@netlify
Copy link

netlify bot commented Oct 11, 2023

Deploy Preview for coderdojobraga-web canceled.

Name Link
🔨 Latest commit 184fe48
🔍 Latest deploy log https://app.netlify.com/sites/coderdojobraga-web/deploys/6526aa6035549e0007e314f6

@netlify
Copy link

netlify bot commented Oct 11, 2023

Deploy Preview for coderdojobraga-app ready!

Name Link
🔨 Latest commit 184fe48
🔍 Latest deploy log https://app.netlify.com/sites/coderdojobraga-app/deploys/6526aa60af99d70008237b36
😎 Deploy Preview https://deploy-preview-199--coderdojobraga-app.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Oct 11, 2023

Deploy Preview for coderdojobraga-blog canceled.

Name Link
🔨 Latest commit 184fe48
🔍 Latest deploy log https://app.netlify.com/sites/coderdojobraga-blog/deploys/6526aa6061627700083800e4

@netlify
Copy link

netlify bot commented Oct 11, 2023

Deploy Preview for coderdojobraga-maintenance canceled.

Name Link
🔨 Latest commit 184fe48
🔍 Latest deploy log https://app.netlify.com/sites/coderdojobraga-maintenance/deploys/6526aa6018c7330008d857e3

@ruilopesm ruilopesm merged commit a653742 into main Oct 11, 2023
14 of 15 checks passed
@ruilopesm ruilopesm deleted the rl/fix-availability-update branch October 11, 2023 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app App related contributions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants